Skip to content

fixed getImages RuntimeError when using boilerpipe-1.2.0.jar built… #31

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
benpryke wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fixed getImages RuntimeError when using boilerpipe-1.2.0.jar built… #31

benpryke wants to merge 1 commit into from

Conversation

@benpryke
Copy link

@benpryke benpryke commented May 22, 2015

…from the latest source

The error that has been fixed was:

>>> from boilerpipe.extract import Extractor
>>> extractor = Extractor(extractor='ArticleExtractor', url='some-url')
>>> extractor.getImages()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Program Files (x86)\Python 2.7.3\lib\site-packages\boilerpipe\extract\__init__.py", line 84, in getImages
    } for image in images
  File "C:\Program Files (x86)\Python 2.7.3\lib\site-packages\jpype\_jcollection.py", line 32, in __next__
    return next(self.iterator)
RuntimeError: No matching overloads found. at native\common\jp_method.cpp:121

This occurred when using a version of boilerpipe-1.2.0.jar built from the most recent source.

@benpryke benpryke mentioned this pull request May 22, 2015
Closed
@mdfahad777
Copy link

mdfahad777 commented Aug 5, 2016

Ki koitam tor to mone koile kicho likhi dis.

@tuxdna
Copy link
Collaborator

tuxdna commented Jan 13, 2017

@Ninjakannon If this PR is still applicable, please resolve the conflicts and update.

Also please add a test case so this can be detected in future.

@benpryke
Copy link
Author

benpryke commented Jan 15, 2017

@tuxdna There are no conflicts and as far as I can tell, this repo has no test cases; it is unclear to me how to best go about adding them.

@tuxdna
Copy link
Collaborator

tuxdna commented Jan 15, 2017

@Ninjakannon Oh right there are no conflicts. Perhaps I mistook with other PRs in this project.

What would be best way to replicate the issue, for which this PR is submitted?

@benpryke
Copy link
Author

benpryke commented Jan 15, 2017 •
edited
Loading

@tuxdna Although I haven't used this for a while, I believe my original merge request comment (at the top) illustrates the bug. To replicate, one must pull the repo, compile boilerpipe-1.2.0.jar from source, extract a webpage, and call the getImages method.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@benpryke @mdfahad777 @tuxdna @misja @Caimany